Skip to content

Ps3eye fixes in Windows and OSX.#162

Merged
thp merged 3 commits into
thp:masterfrom
psmoveservice:ps3eye_fixes
May 29, 2015
Merged

Ps3eye fixes in Windows and OSX.#162
thp merged 3 commits into
thp:masterfrom
psmoveservice:ps3eye_fixes

Conversation

@cboulay
Copy link
Copy Markdown
Contributor

@cboulay cboulay commented Mar 11, 2015

This replaces PR #152

-tracker fails gracefully when PS3EYE specified at compile time but not plugged in.
-exposure and gain parameters passed through to PS3EYE when using PS3EYEDriver. This greatly improves the performance of the tracker in well-lit environments.

@thp
Copy link
Copy Markdown
Owner

thp commented Mar 15, 2015

Oh, by the way - you can force-push changes over an existing branch, thereby updating a pull request and replacing the commits, no need to always open and close pull requests, as it makes it hard for me to follow along where we are at the moment (I think I just commented in a commit in a closed pull request).

@cboulay
Copy link
Copy Markdown
Contributor Author

cboulay commented Mar 15, 2015

Sorry about that. I'll look into force-push. Do you want me to work on the closed PR or is it OK to keep this one for this time?

@thp
Copy link
Copy Markdown
Owner

thp commented Mar 15, 2015

Yes, let's keep this one for this time. No problem, just a suggestion for the future :)

Setting red and blue balance values does not work well with PS3EYEDriver. Comment out for now.
Comment thread src/tracker/psmove_tracker.c Outdated
Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This shouldn't happen anymore, now that you return NULL above for when tracker->cc is NULL.

@cboulay
Copy link
Copy Markdown
Contributor Author

cboulay commented May 27, 2015

This should be ready now.

@thp
Copy link
Copy Markdown
Owner

thp commented May 29, 2015

Looks good now, thanks a lot :)

thp added a commit that referenced this pull request May 29, 2015
Ps3eye fixes in Windows and OSX.
@thp thp merged commit 33a764d into thp:master May 29, 2015
@cboulay cboulay deleted the ps3eye_fixes branch May 29, 2015 17:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants